Skip to content

slick-greeter: a 'user interface' for lightdm #876

Merged
merged 2 commits into from
Sep 14, 2018
Merged

Conversation

thomas
Copy link
Collaborator

@thomas thomas commented Aug 22, 2018

Homepage:

https://github.com/linuxmint/slick-greeter

Notes:

The major changes are adjustments of the display-size and
ordering of the sessions defined on mariux.

Homepage:

https://github.com/linuxmint/slick-greeter

Notes:

The major changes are adjustments of the display-size and
ordering of the sessions defined on mariux.
show-a11y=false
# since german is hammered into /etc/local/X11_xorg.conf, we can skip this
# (also 'de nodeadkeys' looks silly).
show-keyboard=false
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, us is our default, since we do not use /etc/X11/xorg.conf anymore. /etc/local/X11_xorg.conf is the local configuration of each system. So the user chose the keyboard layout during install time.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the description was a bit missleading, show-keyboard=false disables the display of a non-functioning keyboard layout chooser, when on, you either get a nice keyboard-icon plus an 'us', or when overriden (hammered...) with 'de nodeadkeys'.
In ascii-art:

[::_::] us   OR   [::_::] de   nodeadkeys

Maybe we should start trusting X to do the layout job on its own?

local slickgreeterconf=/etc/lightdm/slick-greeter.conf
mkdir -vp ${D}/etc/lightdm

tail -32 ${S}/README.md | sed -e 's/^\s*//' > ${D}/${slickgreeterconf}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment, that this is fragile?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better don't do it. I suggest to create the whole file verbatim and refer to the README.md insetad of copying the comments.

# LightDM GTK+ Configuration
# see https://github.com/linuxmint/slick-greeter/blob/master/README.md for available options

draw-user-backgrounds=false
play-ready-sound=false
...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So you opt for a heredoc in the bee file?
My proposal would have been to defuse the content from the md with more pound signs at the beginning :)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Complete file into the bee file looks better to me than to assume anything about the README.md which is targeted an human readers. We don't need comment-lines in the file. A single reference would be enough and wouldn't need to be updated.

## by default this may be 'make install DESTDIR="${D}"'

mee_install_post() {

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing spaces.

case ${dsb} in
xfce*)
ln -v -s xfce.svg ${badgedir}/${dsb}.svg
;;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, sorry for xfce4_hidpi. Now I see it.

@pmenzel pmenzel changed the title slick-greeter: an 'user interface' for lightdm slick-greeter: a 'user interface' for lightdm Sep 4, 2018
@donald donald merged commit 64a7895 into master Sep 14, 2018
@donald donald deleted the add-slick-greeter-1.2.2 branch September 14, 2018 08:46
@thomas
Copy link
Collaborator Author

thomas commented Sep 14, 2018

Hi Donald,
Well this is more water on your 'bundle things into one pull request' argument.
To enable the slick greeter, we need also to merge 'lightdm: low dependency login/display manger' (PR #875, ¯_(ツ)_/¯ )

Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants